Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add appserving dashboard #200

Merged
merged 3 commits into from
Aug 30, 2023
Merged

add appserving dashboard #200

merged 3 commits into from
Aug 30, 2023

Conversation

robertchoi80
Copy link
Contributor

기존의 View-container 대시보드에서 상단의 variable 부분을 hide하고, 몇몇 패널을 삭제한 앱서빙용 대시보드 추가합니다.
앱서빙 화면에서 바로가기 버튼을 통해 접속할 대시보드입니다.

@intelliguy
Copy link
Contributor

상단의 파드통계부분도 빼야할꺼같습니다.
스크린샷 2023-08-30 오후 4 23 02

@robertchoi80
Copy link
Contributor Author

robertchoi80 commented Aug 30, 2023

@intelliguy 연결할 때 사용자 namespace를 지정해서 요청해서, 상단 통계도 해당 namespace에 대해서만 나오더라구요. 거의 1개 아니면 2개 이렇게 표시되는.. 그래서 유효한 패널 같아서 지우지 않았습니다.

@ktkfree
Copy link
Contributor

ktkfree commented Aug 30, 2023

helm-charts 는 변경시 기존 코드에 영향이 없으려면 반드시 버전을 올리는 것으로 논의되었던것으로 기억합니다. 이 케이스는 lma-addons 의 버전을 올려야 하지 않을까요?

@robertchoi80
Copy link
Contributor Author

@ktkfree 버전 올려서 커밋했습니다

@ktkfree
Copy link
Contributor

ktkfree commented Aug 30, 2023

@ktkfree 버전 올려서 커밋했습니다

넵, 이 버전이 적용되기 위해서는 이 chart 를 호출하는 decapod-base-yaml 이나 decapod-site 에서도 버전업이 되어야 겠습니다.

@robertchoi80
Copy link
Contributor Author

robertchoi80 commented Aug 30, 2023

@ktkfree 네, 이 대시보드를 일단 dev 사이트에만 넣을지 아님 바로 전체 적용해도 괜찮을지 고민 중입니다.

-> 일단, dev 먼저 적용해보는 게 좋을 듯 하여, value로 분기시킬 수 있도록 대시보드 prefix를 'appserve'로 변경해서 커밋했습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants